Revert "Remove _gtk_box_get_children"

This reverts commit 572e9a04027e213082a5b257e5d662a5daa32667.

_gtk_box_get_children was not doing exactly the same than
gtk_container_get_children does, because the latter uses the forall
implementation of GtkBox that takes into account the children pack mode while
the former just iterated the list of children. This broke the order of
the buttons in a GtkButtonBox when they were packaged with PACK_END.

placessidebar: Increase padding

This should bring us back to the look of 3.18.

https://bugzilla.gnome.org/show_bug.cgi?id=761791

treeview: Make sure the editing widget's top left is always visible

https://bugzilla.gnome.org/show_bug.cgi?id=761838

Adwaita: Don't grow flat entries

Entries in treeviews and similar embedded situations don't really
work well if their height is forced to be big. Take out the min-height
for these.

https://bugzilla.gnome.org/show_bug.cgi?id=761686

cannot not -> must

https://bugzilla.gnome.org/show_bug.cgi?id=761826

placesview: Fix some minor issues

Make the button not grow too big, and ensure that the actionbar
has a background.

https://bugzilla.gnome.org/show_bug.cgi?id=761792

Adwaita: tone down treeview separators

Fix Quartz clipboard image retrieval.

It does no good to iterate through a series of mime types to call a
function when the eventually-called function,
_gtk_quartz_get_selection_data_from_pasteboard() in this case, gives the
wrong answer and stops the iteration on all but one especially if that
one isn't first.

The one is "image/tiff" and the quartz pasteboard function will return
any image type Quartz knows about for it, so lose the iteration and use
only "image/tiff".

Adwaita: fix bogus selector...

...ruining the indeterminate check animation

Adwaita: animate check and radios

mir: Add C++ guards

Avoid missing symbols when compiling C++ including the GDK Mir backend
header.

https://bugzilla.gnome.org/show_bug.cgi?id=761829

notebook: fix arrow states

Some code was confusing 0 with ARROW_NONE, causing us to not properly
update the state of the arrow gadgets.

inspector: Don't use unused variables

Windows-specific refactorings broke this.

win32: Move init code into windows-specific code

Avoids warnings about undefined functions on non-windows.

win32 theme: Add a GtkWin32Theme object

This object wraps a HTHEME and takes care of recreating it on theme changes.

win32: Capture WM_THEMECHANGED

We're not doing anything useful with it yet. This will follow in the coming commits.

win32: Remove XP-specific code

We don't support XP anymore.

win32: Remove unused functions / header includes

win32: Always pass a window to the default event filters

This has to happen or the event filter cannot identify what type
of display the event is from (ie if it's an XEvent, a broadway
event or a win32 MSG.

testsuite: No warnings on type conversions please

long is 32bit on win64, so it's int cast to pointer of different size, and gcc doesn't like that.

filechoosernativewin32: Remove unused variables

imcontextsimple: Remove unused variable on win32

testgtk: Compile without warnings on win32

tests: Remove unused variable

win32 theme: Get rid of deprecated style properties

That's pretty easy, because we can just delete them. The theme is broken either way.

win32 theme: Replace pseudoclasses with undeprecated ones

win32 theme: Do a quick'n'dirty search/replace

Try to replace the old style names with then new style names.

The theme is still entirely way too broken. But at least one can see something.

win32: Remove _gdk_display usge from dnd code

win32: Make cursor code work without _gdk_display

win32: Stop using _gdk_display global in device code

Use gdk_device_get_display() instead.

win32embedwidget: Mark deprecated sections

filechoosernativewin32: Silence gcc

filechoosernativewin32: Add missing return type

Looks like a line accidentally was deleted there.

filechoosernativewin32: Define MSC_VER at top of file

The headers we include might include windows headers which define _MSC_VER already.

printoperation-win32: Silence gcc

mountoperation: Silence gcc

Also, printf()ing a GPid should be forbidden.

immodule: Stop gcc from complaining

win32: Remove deprecation warnings in dnd code

win32: Remove deprecation warnings in devicemanager code

win32: Use g_set_object()

... instead of a hand-rolled version that does the same thing.

win32: Fix empty body warning

gtk-demo: Add more foreign drawing examples

Add a notebook with an active tab.
Based on a patch by Caolan McNamara.

gtk-demo: attempt at a notebook with active box-shadow in tab

testgtk: Remove an unused variable

gtk-demo: Add a menu to foreigndrawing

and demo that the block arrows require GTK_STATE_FLAG_DIR_LTR or
GTK_STATE_FLAG_DIR_RTL set to appear

https://bugzilla.gnome.org/show_bug.cgi?id=761828

Updated Brazilian Portuguese translation

overlay: set css name

frame: Set border gadget state after creating it

spinbutton: Free the gadget in finalize

css node tree: Fix crash

This function would return "" whenever the state of the css node was
unset, causing a crash when selecting the css node tree from the
sidebar.

gtk-demo: Improve the tag implementation

Add margins and make it so that clicking on margins next to
links does not activate them.

text view: Improve tag pointer tracking

A problem that has been observed in polari is that links in tags
are clickable all the way into the margin. This problem is caused
by gtk_text_view_get_iter_at_position ignoring the return value
of pango_layout_xy_to_index. Instead, pass it back as a boolean
return value. This is technically an API break, but we've allowed
ourselves to change return types from void to gboolean before.

gtkmagnifier: Avoid a crash if the inspected widget is finalized

If the inspected widget is finalized and we then try to disconnect
from its signals we'll crash.

https://bugzilla.gnome.org/show_bug.cgi?id=761775

A few more documentation fixes

Various documentation fixes

Always have Since: annotations at the very bottom, use the correct
ClassName::signal-name/ClassName:property-name syntax, fix a few typos
in type names, wrong function names, non-existing type names, etc.

gtk: Fix some return-values annotations (transfer none/full)

win32: properly handle show and focus

The first time a window is shown we should always call SW_SHOWNORMAL.
Understand whether to call SW_SHOW or SW_SHOWNORMAL and the specific
ones for the temporary windows depending on IsWindowVisible.
This also fixes the problem when calling gtk_window_present and
the window is snapped to the left or right of the screen.

This patch is based on the patches provided by Yevgen Muntyan
and Aleksander Morgado.

https://bugzilla.gnome.org/show_bug.cgi?id=698652

win32: on focus restore the window if iconified

If the window is iconified we want to restore the window
to get the proper size instead of showing it normal which
would change the size of the window.

https://bugzilla.gnome.org/show_bug.cgi?id=698652

Updated Kazakh translation

inspector: Avoid crash in magnifier

We can't use gtk_widget_draw on widgets that are not allocated.

Add css node tests for box packing

More tests to ensure that CSS node order is always physical
left-to-right.

image: _get_style_context never returns NULL

Make gtk_icon_helper_invalidate private

Add and use gtk_icon_helper_invalidate_for_change

iconhelper: Save whether the surface is symbolic

stylepropertyimpl: Use GTK_CSS_AFFECTS_SYMBOLIC_ICON

Things like color affect symbolic icons, but not colored icons, while
other css properties like -gtk-icon-effect affect colored icons, but not
symbolic ones.

csstypes: Add GTK_CSS_AFFECTS_SYMBOLIC_ICON

cssstylechange: Add helper function to print change

So I don't have to print both styles to the console, paste them both
into a file and then run diff on the 2 files anymore.

iconhelper: Un-duplicate some code

Remove more unnecessary redraws

The call to gtk_widget_set_state_flags immediately before these already
queues a redraw/allocate/resize in case they have to be queued.

Adwaita: No rounded corners for fullscreen

If the theme has rounded corners for fullscreen, we don't tell the
window manager that we are now fully opaque, which then makes things
less efficient than they should be.

https://bugzilla.gnome.org//show_bug.cgi?id=761571

box gadget: Fix allocation without baseline

When we don't get a baseline passed in, we want to basically
center the children inside the allocation. There was an attempt
in the code to do 'internal baseline alignment', but it had the
side effect of moving the contents to the top when we don't get
a baseline passed in. Remove it for now, this needs some more
infrastructure to do properly.

https://bugzilla.gnome.org/show_bug.cgi?id=761363

widget-factory: Update levelbar example

Set reasonable level offsets.

Updated Spanish translation

Updated Spanish translation

Fix levelbar css node tests

These were affected by the recent change to level offset handling.
At the same time, make the test files more realistic by updating
the level offsets when we set a custom range.

level bar: Document buildable syntax

Updated Polish translation

level bar: Fix offset behavior

We had some odd special-casing for the lowest and highest offset
that did not quite work. The new rule is simple: If the value
is between offset n-1 and n, it gets the style for offset n.

https://bugzilla.gnome.org/show_bug.cgi?id=761416

level bar: Improve documentation

The docs were not explaining at all what happens to existing
level offsets when the min- or max-value of a level bar are
changed.

level bar: Make the full offset official

We are adding 3 offsets, not just two. Add a define for the
third one, and mention it in the docs.

Updated Slovak translation

wayland: rework buffer management code (3 changes)

There are a couple of issues with the way that buffers are handled in
wayland in right.  These issues mean that:

 - buffers can get leaked at a fairly fast clip under the right
   conditions. This leads to the OOM killer kicking in and
   gnome-shell and gnome-terminal (for instance) showing memory
   usage in the high gigabytes range.

 - drawing can happen to a shared memory buffer at the same time
   the compositor is reading out the pixels.  This can lead to
   glitching in drawing and other undefined behavior by the compositor.

This changeset reworks how buffer management is done in the code to try
to address both problems.

The first change (commit 2c300081) addresses the leak by dropping code
that has an unchecked cairo_surface_reference call.  The code is dropped
rather than fixed, because it has a more serious issue: it's overarching
purpose is to deal with shared memory buffer contention with the
compositor, but it does it in a racy way and so fails at that mission.

The second change (commit 40e91195a) moves what layer of the code buffer
release events are handled. This is an organizational change in the
code, with no functional changes, but it's important for the last change
in the changeset.

The last change (commit c80dd549) adds back code for dealing with shared
member buffer contention in a race free way. The new code is careful to
never reuse a buffer that hasn't been explicitly released by the
compositor.

wayland: stage uncommitted changes to dedicated buffer

Right now we use one buffer for both staged changes (freshly painted
changes waiting for the frame clock to send to the compositor) and
committed changes (changes actively being read by the compositor
process). This creates a problem in the event we need to stage updates
at the same time the compositor is processing committed updates: we
can't change what the compositor is actively processing.

The current solution for handling this contention is to allocate a
temporary buffer on the spot at the time the updates are staged, and to
copy that buffer back to the shared buffer later.  The problem, though,
is that the copy to the shared buffer currently happens as soon as
the updates are finished being staged, not when the shared buffer is
done being processed by the compositor.

In order to address that problem, this commit changes the code to always
stage changes to a dedicated staging buffer.  The staging buffer is
used exclusively by the client until the client is done with it, and then
once that staging buffer is committed, the client never writes to that
buffer again.  If the client needs to stage new updates, it allocates a
brand new staging buffer, draws to it, and back fills the undrawn parts
of the buffer from a copy of the contents of the committed buffer.

As an optimization, the compositor has the option of releasing the
committed buffer back to the client.  If it does so before the client
needs to stage new updates, then the client will reuse the buffer
for staging future updates.  This optimization prevents having to allocate
a new staging buffer and the associated cost of back filling
that new buffer with a readback of the committed buffer.

https://bugzilla.gnome.org/show_bug.cgi?id=761312

wayland: don't handle buffer release centrally

Right now we handle buffer releases coming from the
compositor in a central place. We add a listener when
first creating the shared buffers.

This is problematic because a buffer can only have
one listener on it at once so users of the buffer
can't get notified when it's released.

This commit moves the buffer listener code from the
centrally managed display code to the cursor and window
code.

https://bugzilla.gnome.org/show_bug.cgi?id=761312

wayland: always return FALSE from begin_paint

The client and compositor share access to the window
pixel buffers. After the client hands off (commits)
the buffer to the compositor it's not supposed to write
to it again until it's released by the compositor.

The code tries to deal with this contention by allocating
a temporary buffer and using that in the mean time. This
temporary buffer is allocated by a higher layer of the code
when begin_paint returns TRUE. Unfortunately, that layer of
the code has no idea when the buffer is released, so it ends
up blitting the temporary buffer back to the shared buffer
prematurely.

This commit changes begin_paint to always return FALSE.

A future commit will address the contention problem in
a different way.

https://bugzilla.gnome.org/show_bug.cgi?id=761312

wayland: use g_clear_pointer when destroying cairo surfaces

There are a few places where we destroy a cairo surface and
then nullify it. This commit changes those to use
g_clear_pointer instead.

It also drops a cairo_surface_finish call that is unnecessary

https://bugzilla.gnome.org/show_bug.cgi?id=761312

wayland: rename cairo surface user data key to be more specific

This commit renames the key name to be more specific for clarity.

https://bugzilla.gnome.org/show_bug.cgi?id=761312

wayland: move server proxy objects to substructure

This commit moves the server proxy objects to a substructure
for clarity.

https://bugzilla.gnome.org/show_bug.cgi?id=761312

wayland: rename window->surface to window->wl_surface

The name surface is really overloaded when dealing
with wayland windows.

To alleviate ambiguity, this commit changes the name
of the "surface" and "subsurface" members to have
a wl_ prefix.

https://bugzilla.gnome.org/show_bug.cgi?id=761312

Fix stylecontext tests

This was broken by f7ec9c98ef0ef8740c93f96af9d971b0211118c1,
since type names are no longer used at all in CSS matching.

Update CSS docs regarding type names

We no longer use type names at all.

cssselector: Remove type references

We don't use types anymore.

cssmatcher: Remove type argument from name matcher

We no longer use types when matching names.

inspector: Drop type name from node list

It's not used anymore in the CSS, so people don't need to see it.

cssimageradial: Only allow at "<position>" after other props

Don't allow syntax like
  at top left circle
but follow the spec about requiring the at <position> right before the
comma.

This is porbably because
  circle at 10px 10px
could be interpreted as
  circle 10px at 10px
with the now disallowed syntax, too.

Test included.

cssimageradial: Only emit one error for wrong position

Test included.

css tests: Fix style tests

Commit d0fd7990c3f2c738f86185418ccd76b14da8a776 changed
the location of builtin themes, breaking the loading of
the empty theme in test-css-style.

css tests: Fix expected output

This is fallout from f7ec9c98ef0ef8740c93f96af9d971b0211118c1.

css: Convert numbers with GTK's dpi

Don't hardcode 96 for dpi, but instead use the value of the -gtk-dpi
property (that mirrors the GdkScreen's dpi if it wasn't set explicitly).

This makes these values scale when the large font setting in
control-center is enabled.